Skip to content

MDEV-39928 Fix GitLab CI centos9 job failure#5198

Open
FarihaIS wants to merge 1 commit into
MariaDB:10.11from
FarihaIS:mdev-39928
Open

MDEV-39928 Fix GitLab CI centos9 job failure#5198
FarihaIS wants to merge 1 commit into
MariaDB:10.11from
FarihaIS:mdev-39928

Conversation

@FarihaIS

@FarihaIS FarihaIS commented Jun 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

The centos9 job in .gitlab-ci.yml fails because it uses yum-builddep -y mariadb-server to install build dependencies, but the mariadb-server source package has been removed from CentOS Stream 9 repositories. This affects all branches that define the centos9 job (main, 10.11, 11.4, 11.8, 12.3).

Fix:

  • Enable the CRB repository and replace yum-builddep -y mariadb-server with explicit build dependency installation
  • Add liburing-devel for io_uring support

Release Notes

N/A

How can this PR be tested?

Run GitLab CI on this branch and verify the centos9 job passes. The build should complete successfully.

Before this change:

Screenshot 2026-06-09 at 15 56 29 
$ yum-builddep -y mariadb-server
enabling baseos-source repository
enabling appstream-source repository
enabling extras-common-source repository
enabling epel-source repository
enabling epel-cisco-openh264-source repository
CentOS Stream 9 - BaseOS - Source               1.9 MB/s | 698 kB     00:00    
CentOS Stream 9 - AppStream - Source            3.4 MB/s | 1.4 MB     00:00    
CentOS Stream 9 - Extras packages - Source       29 kB/s |  15 kB     00:00    
Extra Packages for Enterprise Linux 9 - x86_64   18 MB/s | 4.3 MB     00:00    
Extra Packages for Enterprise Linux 9 openh264   75  B/s | 1.2 kB     00:15    
no package matched: mariadb-server
Error: Some packages could not be found.

After this change:

Basing the PR against the correct MariaDB version

  • This is a minor fix and the PR is based against the earliest maintained branch in which the bug can be reproduced.

Copyright

All new code of the whole pull request, including one or several files that are either new files or modified ones, are contributed under the BSD-new license. I am contributing on behalf of my employer Amazon Web Services, Inc.

@gemini-code-assist

gemini-code-assist Bot commented Jun 8, 2026

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@FarihaIS FarihaIS marked this pull request as ready for review June 8, 2026 21:44
@gkodinov gkodinov added the External Contribution All PRs from entities outside of MariaDB Foundation, Corporation, Codership agreements. label Jun 9, 2026
gkodinov
gkodinov requested changes Jun 9, 2026
View reviewed changes

@gkodinov gkodinov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution! This is a preliminary review.

I have tried looking into when CentOS9 lost its mariadb server package, but couldn't find anything.

Nevertheless, this sounds like a bug fix to me, so I'd like to request that you rebase this towards 11.8 (this seems to be the earliest release train that has centos 9).
FWIW, 10.11 has CentOS7, but I'm not sure if that one is affected.

@CLAassistant

CLAassistant commented Jun 9, 2026 •
edited
Loading

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@FarihaIS FarihaIS changed the base branch from main to 10.11 June 9, 2026 21:00
@FarihaIS FarihaIS marked this pull request as draft June 9, 2026 21:02
@FarihaIS FarihaIS marked this pull request as ready for review June 9, 2026 22:57
@FarihaIS

FarihaIS commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

@gkodinov thank you for the feedback! It seems 10.11 and 11.4 also have the centos9 job, so I have rebased my commit on 10.11 and updated the PR to reflect this.

Could you please let me know if everything looks okay now?

@FarihaIS FarihaIS requested a review from gkodinov June 9, 2026 22:57
gkodinov
gkodinov approved these changes Jun 10, 2026
View reviewed changes

@gkodinov gkodinov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Please stay tuned for the final review.

@gkodinov gkodinov requested a review from grooverdan June 10, 2026 07:43
@grooverdan

grooverdan commented Jun 10, 2026

Copy link
Copy Markdown
Member

Note our CI dependency list is https://github.com/MariaDB/buildbot/blob/dev/ci_build_images/centos.Dockerfile.

eigen3-devel should be removed (decided not to use it for vector). Some I can't explain (subversion). check-devel was for galera. Liburing is because both are supported (one falls back to the other if not present). If you have anything you want to update I'll look again in 12+ hours.

@FarihaIS
MDEV-39928 Fix GitLab CI centos9 job failure
a31a802 
The centos9 job uses yum-builddep -y mariadb-server to install build
dependencies, but the mariadb-server source package has been removed
from CentOS Stream 9 repositories. Replace with explicit installation of
the required build dependencies.

All new code of the whole pull request, including one or several files
that are either new files or modified ones, are contributed under the
BSD-new license. I am contributing on behalf of my employer Amazon Web
Services, Inc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gkodinov gkodinov gkodinov approved these changes

@grooverdan grooverdan Awaiting requested review from grooverdan

Assignees

@grooverdan grooverdan

Labels

External Contribution All PRs from entities outside of MariaDB Foundation, Corporation, Codership agreements.

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@FarihaIS @CLAassistant @grooverdan @gkodinov